Parallel test (#303)

* run vtesto jobs in parallel.

with the configure flow and "make check-vtesto".
This will be utilized on travis.

* add missing gitignore for new valgrind logs.

fix pcx test to check output it generates! (#302)

update appveyor for Qt 5.12.1 (#301)

* update appveyor builds.

archive msvc2017,nmake, Qt 5.12,x86.
switch all msvc2017 builds to Qt 5.12.

* work around qmake 5.12.1 bug.

reorder builds so release candidate is done first.

archive macos with Qt 5.12.1 (#300)

instead of 5.9.7.

Merge pull request #293 from tsteven4/trackf

modernization of track filter

Merge pull request #271 from gromit1811/fit-trkseg

Start new track segments on Garmin Fit timer events (start/stop/...)

Relocate xcsv format functions. (#297)

Many funtions for the xcsv format were in csv_util.cc
instead of xcsv.cc.  This commit moves the xcsv format
specific functions to xcsv.cc, and leaves the possibly
shared utility functions in csv_util.cc.

xcsv.h is added to communicate shared xcsv style handling
with vecs.cc.

Added support for USR version 5 and 6 and corrected USR version 4 support. (#268)

* Minor changes to address GCC 7.3.0 warnings generated by flag -Wunused-but-set-variable.

Moved declaraions/initialization of rp_backup and era inside #ifdef _NL_CURRENT conditionals.

* Added { ... } to for at the end of zip64local_SearchCentralDir() so that if-break statements inside for loop.

* Removed declaration and commented out references to startlat, startlon, and starttime in fit_parse_data() due to GCC 7.3.0 warning from
flag -Wunused-but-set-variable.

* Added initial support for Lowrance USR 5 and 6 formats.

* Significant changes to handle multiple USR4 versions and add USR5 and USR6 support.

* Remove extraneous debug NL when no debug specified.

* Updated USR4 reference file because of increased accuracy in converting lat/log values.

* Documentation on some ICON mappings.

* Added USR5 and USR6 to validation checks.

* Added reference files for USR5 and USR6 formats.

* Cleaned up some debug output.

* Continued work to consolidate all Lowrance USR processing into a single file.

* Eliminated lowranceusr4.cc and restuctured testing to include USR4, USR5, and USR6 formats.

* Changes associated with adding support for USR 5 and USR 6.

* Eliminated lowranceusr4.cc from build.

* Changed default Output USR Version from 4 to 2 for backward compatibilty.

* Removed USR 4 section from documentation.

* Initial update of XMLDOC for updated data formats.

* iMinor updates to parse_routes.  Added correct v4 and v5 GPX reference files.

* Documentation updates to reflect new capabilities.

* Added Route element information to document and minor code changes to match.

* Doc refinement with rewrite.

* Corrections to USR 4 output.

* Update to correct v4 gpx file.

* Checkin correct v5 gpx file.

* Removed errant NL when no debug specified.

* Resolved conflicts

* Missed this update.

* Revert "Minor changes to address GCC 7.3.0 warnings generated by flag -Wunused-but-set-variable."
at the request of tsteven4 as they have nothing to do with the expanded support.

This reverts commit fb93a19b740edae4dde3b6dcd0546f344122f70d.

* Revert "Added { ... } to for at the end of zip64local_SearchCentralDir() so that if-break statements inside for loop."
at the request of tsteven4 as they have nothing to do with the expanded support.

This reverts commit 189bbe6ea664829412c70079e9ddb9555062a12f.

* Revert "Removed declaration and commented out references to startlat, startlon, and starttime in fit_parse_data() due to GCC 7.3.0 warning from"
at the request of tsteven4 as they have nothing to do with the expanded support.

This reverts commit cc618336888c5cdb8adef87f72026a8c24eff7e8.

* Updated to remove lowranceusr4.cc per suggestion from tsteven4 to support cmake/XCLion.

* Per suggestions from tsteven4
    - deleted define SECSTORADIANS as it was not used
    - ran astylerc

* Replaced use of NULL with nullptr per tsteven4 suggestion.

* Added URLs for reference information.

* Adopted tsteven4 recommendation on eliminating compiler warning.

* Corrections to Waypoint and Trail documentation for USR 4, 5, and 6 formats.

* Address Codacy flagged issues.

* Address Codacy flagged issues.

* Address additional Codact flaged issues.

* Changed DEGREESTORADIANS define from hardcoded value to use (M_PI/180.0)

* Removed Copyright protected material and replaced with authored comments.

* Corrected altitude/depth retension coverting USR4/5/6 to USR2/3

* Added mapping of USR4/5/6 ICON numbers to text to support output to USR 2/3 format.

* Corrected generation of USR4 data file.

* Check for NULL format specific when converting USR 2/3 (colorless) to USR4 (color coded) ICONs.

* Added cases where USR 2/3 embeds color in shortname.

* Corrections to ICON translation from USR v2/3 to USR 4.

* Cleanup USR 2/3 parsing debug.

* Adopted modificatins to USR 4/5/6 time interpretation per suggestion from tsteven4.

* Adopted modificatins to USR 4/5/6 time interpretation per suggestion from tsteven4.

* Changed file header parsing back to int16 for USR format as found sample file with a stream version other than 0.  Assume this
is true for ALL USR formats until determine otherwise.

* Updated reference files after corrections to altitude and time conversion.

* Check for both "nan" and "-nan" to address compiler differences.

* Use -options instead of --options to avoid osx OS issues.

* Set altitude to "unknown_alt" when creating Waypoints from Event ICONs.
Updated reference files to encorporate "unknown_alt" change.
Corrected mis-spelling of "green"

* Change cut options from --compliment to legacy explict field list for OSX OS support.

* Applied codex patch from tsteven4 - thank you.

* Corrected interpretation of elevation in USR2/3.

* Corrected reference files and test script with changes to elevation interpretation.

* More conversions to QString conventions.
Minor correction to trail attribute parsing.

* Editorial cleanup.

* USRv6 uses trail data stream version 5 which employs int32 rather than char for attributes.

* Use invalid ICON value as method to detect HOOK2 generated USR 2/3 files.

* Cleanup time conversions.

bug fixes for xcsv reader. (#295)

For some lines data must be accumulated from several fields
in arbitrary field order.
Previously this was done with global variables.  Now a variable to
hold the accumulated data for a line is constructed/destructed for
each line.

This eliminates the possiblity of unintended communication between
lines.

This also eliminated some unintended communication between the reader
and the writer through global variables that they previously shared
(csv_track, csv_route).

An ordering sensitivty to TRACK_NEW and TRACK_NAME is removed.

For the reader, conditional allocation of route_heads is removed
from xcsv_parse_val, and centralized in xcsv_data_read.
This eliminates some undetected
memory leaks which hid as "still reachable" due to the pointer
being saved in a global variable (csv_track, csv_route).

Undesirable reading of the route_head->Q variable is
eliminated improving encapsulation.  I beleive the intent was
never to decide if the csv_track->Q was empty, i.e. it hadn't been
added to the global track_head yet.  I beleive what was desired was
to decide if csv_track had any waypoints. In any event the
centralization of route_head allocation and head addition makes new
track handling much more straight forward and robust.

The xcsv test case is enhanced to increase coverage.

Mac5120 (#283)

* add generic qt installer for osx.

add travis builds with osx and qt 5.9.7, 5.12.0
update qtci extract-qt-installer from upstream.
add some error detection and debug to travis_install_osx, travis_intall_linux_local.

* catch qtci/install-qt up with upstream.

* try harder with early extract-qt-installer bugs on osx.

* don't source travis qt install scripts.

* fix caching for travis.

* qt installer improvements

validate installs
add debug for failed installs

* update travis osx qt 5.12 build to 5.12.1.

Start new track segments on timer start events

Insert track segment separators when encountering a timer start event in a
FIT file (usually caused by manually pushing the start button). This way,
pauses in a FIT file don't appear as continuous tracks.

Also adapt reference tracks by adding suitable <trkseg> tags to please "make
check"/testo.

Merge pull request #286 from gpsbabel/cl-cleanups

Clion & related cleanups

fix leaks when deleting wpts in filters. (#292)

fix trackfilter merge bug with segments.

enhance trackfilter test cases for better coverage.

refactor trackfilter.

use std::sort, stable_sort instead of qsort.
use QList instead of malloc'd arrays.
minimize the use of duplicated information (track_ct, track_pts, timeless_pts
first_time, last_time), and maintain the remaining
duplicated information (track_list, a possible sorted version of my_track_head).
use auto some more.
eliminate some redundant casting.
eliminate atoi.

in trackfilter remove redundant casts and use auto.

trackfilter bools and case insensitive opts.

refactor track filter.

user visible changes:
Previously it was possible to cause a segmentation fault (due to a double delete)
if all the tracks were filtered out by the stop and start options.
This has been corrected.

Previously UTC time was used with the title option for split operations,
but local time was used with the title option for the pack operations.
Now UTC time is always used with the title option.

Previously the split and sdistance options allowed an option value
consisting of only the units.  This was undocumented.  Now both a number
and the units are required.

Previously the split option documentation stated that units of seconds
would be used if none were specified, however the units were actually required.
The documenation is updated to reflect historical and current usage.

Previously the split and sdistance options allowed the units to precede or
follow the quantity.  The documentation stated the units must follow the
quantity.  Now the units must follow the quantity.

Previously synthesized speeds ignored the fractional part of the track point times,
now the fractional part of the track point times are used in the speed calculation.

refactoring changes:
time_t is replaced where possible by gpsbabel::DateTime or QDateTime.

option value parsing is done with QRegularExpression.

some integer types were converted to bool types.

test cases are expanded for better coverage.

Corrected handling elevation and speed (#291)

* Corrected handling elevation and speed

* Added reference files for testing converting fit-file from Garmin Oregon 700 to gpx-file

* Added test of converting fit-file from Garmin Oregon 700 to gpx-file

add include guard for src/core/optional.h (#290)

clazy cleanup for gui (#289)

Merge pull request #288 from tsteven4/lat_dir_input

enhance xcsv reader to not ignore LAT_DIR,LON_DIR.

Merge pull request #287 from tsteven4/lat_dir

fix up style file OFIELD option processing.

enhance xcsv reader to not ignore LAT_DIR,LON_DIR.

this came up on [Gpsbabel-misc] Globalsat DG-500.

fix up style file OFIELD option processing.

Merge pull request #285 from gpsbabel/atoi

Atoi: rely less on atoi, scanf, other cleanups. Safer parsing in stmsdf in particular.

A moderately uncomfortable series of inadequately tested changes around
signedness of types, mostly to shut hyperactive tool chains. Time will
tell if we fixed or caused more weird cases around trucations and extensions.

Merge branch 'master' into atoi

Merge pull request #284 from gpsbabel/cl-cleanups

CodeLion-inspired and manual cleanups

add support for multiple urls for route headers. (#276)

* add support for multiple urls for route headers.

add support in gpx for gpx/rte/link, gpx/rte/url, gpx/rte/urlname.
add support in gpx for gpx/trk/link, gpx/trk/url, gpx/trk/urlname.

Note that the gpx writer can violate the schema when fprint_xml_chain
is used to echo unrecognized elements from input to output.
This is because the gpx 1.0/1.1 schema requires an xsd:sequence of
elements, i.e. the order of the elements is specified, and any
elements output by fprint_xml_chain won't necessarily be in
the correct order with other elements that are specifically
handled in the gpx writer.

By processing additional gpx elements as listed above on read we
prevent them from causing schema violations on write.  However,
other unrecognized elements can still cause schema violations.

* make common link code common in gpx reader.

Don't #include debug

A near rewrite of the reader of the stmsdf reader to more effectively parse fields as QDate,Times,Floats, etc. Still not very readable, but should be more bulletproof for bad input.

Don't name params in decls in itracku

Sign extension fix in EasyGPS

Remove xstrrstr and BASE_STRUCT from defs.h

Remove write-only variable in compegps.

Drop some unused method protos from CET.

XmlStreamWriter: make single arg methods explicit. Drop unused methods.

Clean up missed spot for ghosting  htmlencoding files

Removing scraggling bits of xhtmlent from build files.

Remove (now) unused xhtmlent table.

Make types in ggv_bin more safe for 32/64 systems.

Default construct to eliminate warning in geojson

Build fix for Cmake/CLion on Mac. Correct handling of frameworks. They're like
libs, but not...

add cppcheck target to GPSBabel.pro. (#280)

* add cppcheck target to GPSBabel.pro.

fix a few cppcheck errors.

* fix a few cppcheck items.

strptime fix is from upstream.

Wrap (use) of variables in #defines.

Warning fix in raymarine

Warning fix in locus

Fix warning in maggeo

Warning fixes in jeeps: move use closer to initialization so that the
test of device is effective, don't set locals before return.

Tame string formatting warning in libusb.

Remove unneeded const qualifier on pass-by-value types.

Replace size() == 0 with empty() when more readable.

Const-ipate jeeps function signature. (Yeah, coz THAT makes that API beautiful...)

Automated cleanups for passing by reference and fixing pointer ownership on calls.

Modernize loop iterators to walk arrays and containes instead of independently
maintained constants.

Modernize C style explicit (void) in decls to (), eliminate return
at end of bunction bodies.

Modernize bool use of true/false.

Make prototype arg names match function arg names.

Pick up a few old NULL->nullptr conversions that are stuck inside __APPLE that
tsteven4 dodged around since he couldn't confirm.

For single argument ctors, mark type explicit.

If there's only one argument in a ctor, mark it explicit to make it less
easy to fall into implicit conversions.

Diff and Terrain in geocaches are integers internally.

generate html coverage reports. (#281)

* generate html coverage reports.

* tweak coverage recipe.

remove obsolete msvc workarounds. (#278)

msvc 2012 and earlier are not supported.
They didn't support c++14 anyway which we currently require.

use QThread::usleep instead of our own function. (#277)

Our test suite doesn't test sleep, but the nmea pause option
can be used on write to verify functionality interactively.

obsolete DEBUG_MEM (#279)

simplify conversion from utm. (#275)

* simplify conversion from utm.

add test cases for Norway and Svalbard exceptions.

* Correct bug in computing latitude band X.

enhance test to cover exceptional width of band X.

update shapelib to 1.4.1 (#274)

enhance sort filter for rtes & trks. (#273)

use bool, true, false. (#272)

add msvc2017 builds to appveyor. (#270)

update docker build script to use latest bionic. (#269)

Merge pull request #266 from tsteven4/clazy1

Incorporate improvements found by Qt oriented code checker clazy

Merge pull request #267 from gpsbabel/gpsbabel-patch-2

Remove leftover code from 'adding class' change

Remove leftover code from 'adding class' change

As tsteven4 points out, No reasing to move is_internal to local and back

fix clazy detected 'use isEmpty() instead'

this is from -Wclazy-isempty-vs-count

fix clazy detected 'Use *Ref() instead'

These are from -Wclazy-qstring-ref, and were automatically fixed.

fix clazy detected 'unused *'

these are from -Wclazy-unused-non-trivial-variable

fix clazy detected 'allocating an unneeded temporary container'

these are from -Wclazy-container-anti-pattern

Also, consistently use range based for loops in csv_util.cc

fix clazy detected 'c++11 range-loop might detach Qt container'

these are from -Wclazy-range-loop

fix clazy detected 'Use multi-arg instead'

these are from -Wclazy-qstring-arg

update travis for xenial (#265)

* update travis to use xenial.

add new travis target to build on xenial.
switch travis coverage generation to xenial.
update codacy upload tool for compatibility with newer java on xenial.

* fix gcov issues with "cannot open notes file"

support kml 2.3 track element. (#264)

* support kml 2.3 track element.

relax ordering requirement for gx:Track when and
coord children.

* fix kml includes and indentation.

Merge pull request #261 from gpsbabel/gpsbabel-patch-2

Update dependencies, notably for <optional>

Replace queues with Qlist in session handling. (#263)

session tracks a possible series of input formats and input filenames.

Merge pull request #262 from tsteven4/tcache

extend travis mac cache timeout.

extend travis mac cache timeout.

Merge pull request #260 from tsteven4/cpp14_msvc

move to c++14 standard.

Merge pull request #259 from tsteven4/queue_bye_bye

eliminate queues in csv style handling.

Update dependencies, notably for <optional>

Merge pull request #256 from tsteven4/rpath

have configure conditionally set rpath.

move to c++14 standard.

eliminate queues in csv style handling.

also:
fix bug with style option encoding.
pass some parameters by const reference.
eliminate some unused time fiddling.

Merge pull request #258 from tsteven4/sortbug

correct sort filter time option to use msecs.

correct sort filter time option to use msecs.

use consistent whitespace in configure comments.

have configure conditionally set rpath.

this may be necessary when using a Qt installation
that was not provided by the system.

Merge pull request #255 from gpsbabel/parallel

Parallelize CLI and GUI make.

Parallelize CLI and GUI make. testo and vtesto are harder.